CircleとRectを別々の配列で管理。まだ改善の余地があります。
p5.js oop
Learning OOP Object Oriented Programming
継承は使っていますが、まだ改善の余地があります。
現状のコード
let objs = []; // Circles
let objs2 = []; // Rects円と四角を別々の配列で管理しています。
問題点
for (let obj of objs) {
obj.update();
}
for (let obj of objs2) {
obj.update();
}
for (let obj of objs) {
obj.display();
}
for (let obj of objs2) {
obj.display();
}同じ処理を2回書いている...これは104(HELL)と同じパターン!
なぜ「CRAZY」なのか
継承でクラスはキレイになったのに、使い方(配列の管理)がまだ古いまま。 次の404で、この問題を解決します。
View Source Code
let W, H, PW, PH;
const PADDING_RATIO = 0.2;
const MAX_SPEED = 10;
let objs = [];
let objs2 = [];
const MAX = 100
class AbstractShape {
constructor() {
this.pos = {
x: random(width),
y: random(height),
};
this.speed = {
x: (Math.random() - 0.5) * MAX_SPEED,
y: (Math.random() - 0.5) * MAX_SPEED,
};
this.acceleration = 0.1;
}
update() {
this.pos.x += this.speed.x;
this.pos.y += this.speed.y;
if (this.pos.x < 0 + PW) {
this.speed.x += this.acceleration;
} else if (this.pos.x > W - PW) {
this.speed.x -= this.acceleration;
}
if (this.pos.y < 0 + PH) {
this.speed.y += this.acceleration;
} else if (this.pos.y > H - PH) {
this.speed.y -= this.acceleration;
}
}
display() {
// implement in subclass
}
}
class Circle extends AbstractShape {
display() {
stroke(0);
fill(0);
ellipse(this.pos.x, this.pos.y, 10, 10);
// text(["1:", Math.floor(this.pos.x), Math.floor(this.pos.y)], this.pos.x + 10, this.pos.y + 10);
}
}
class Rect extends AbstractShape {
display() {
stroke(0);
fill(0);
rectMode(CENTER)
rect(this.pos.x, this.pos.y, 10, 10);
// text(["2:", Math.floor(this.pos.x), Math.floor(this.pos.y)], this.pos.x + 20, this.pos.y + 20);
}
}
// main
function setup() {
createCanvas((W = windowWidth), (H = windowHeight));
PW = W * PADDING_RATIO;
PH = H * PADDING_RATIO;
for (let i = 0; i < MAX; i++) {
objs.push(new Circle())
}
for (let i = 0; i < MAX; i++) {
objs2.push(new Rect())
}
}
function draw() {
background(255);
for (let i = 0; i < objs.length; i++) {
const obj = objs[i];
obj.update();
}
for (let i = 0; i < objs2.length; i++) {
const obj = objs2[i];
obj.update();
}
for (let i = 0; i < objs.length; i++) {
const obj = objs[i];
obj.display();
}
for (let i = 0; i < objs2.length; i++) {
const obj = objs2[i];
obj.display();
}
}